Skip to content

fix: use brand color for error toast#1023

Merged
jvsena42 merged 3 commits into
masterfrom
fix/error-toast-color
Jun 17, 2026
Merged

fix: use brand color for error toast#1023
jvsena42 merged 3 commits into
masterfrom
fix/error-toast-color

Conversation

@jvsena42

@jvsena42 jvsena42 commented Jun 17, 2026

Copy link
Copy Markdown
Member

Fixes #1009

This PR switches the .error toast color from the crimson red accent to the brand orange so it matches the design prototype.

Description

The error toast type previously used the red accent (#E95164), a crimson that does not appear in the prototype. The prototype's failure toasts ("Spending Balance Setup Failed", "Instant Payments Unavailable") all use the brand orange (#FF4400), so error toasts now use the brand color too.

The separate warning toast branch is intentionally left in place even though it currently resolves to the same brand color, so a distinct color can be reintroduced for errors in the future without reworking the mapping.

Design reference: Figma prototype

Preview

Screen_recording_20260617_142317.webm

QA Notes

Manual Tests

  • 1. Trigger an error toast (e.g. failed payment or setup failure): toast tint shows brand orange, not red.

Automated Checks

  • N/A

@jvsena42 jvsena42 self-assigned this Jun 17, 2026
@jvsena42
jvsena42 requested a review from ovitrif June 17, 2026 17:29
@greptile-apps

greptile-apps Bot commented Jun 17, 2026

Copy link
Copy Markdown

Greptile Summary

This PR fixes the error toast color by changing it from Colors.Red (#E95164) to Colors.Brand (#FF4400) to match the Figma design prototype. A changelog fragment for issue #1009 is also added.

  • ToastView.kt: The tintColor() mapping for Toast.ToastType.ERROR is updated from Colors.Red to Colors.Brand; WARNING already mapped to Colors.Brand, so both types now share the same color — the PR notes this is intentional.
  • changelog.d/next/1009.fixed.md: A one-line changelog entry is added describing the fix.

Confidence Score: 5/5

Safe to merge — the change is a one-line color swap in a single composable function with no logic impact.

The only changed behavior is the toast tint color for ERROR toasts. The mapping is trivially correct and the rest of the component is untouched. The one observation — that WARNING and ERROR now share the same color — is acknowledged and intentional per the PR description.

No files require special attention.

Important Files Changed

Filename Overview
app/src/main/java/to/bitkit/ui/components/ToastView.kt Single-line change swapping Colors.Red to Colors.Brand for the ERROR toast tint; WARNING already used Colors.Brand so both now share the same color.
changelog.d/next/1009.fixed.md New changelog entry describing the error toast color fix for issue #1009.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    T[Toast.tintColor] -->|SUCCESS| G[Colors.Green]
    T -->|INFO| B[Colors.Blue]
    T -->|LIGHTNING| P[Colors.Purple]
    T -->|WARNING| BR[Colors.Brand #FF4400]
    T -->|ERROR| BR
    style BR fill:#FF4400,color:#fff
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    T[Toast.tintColor] -->|SUCCESS| G[Colors.Green]
    T -->|INFO| B[Colors.Blue]
    T -->|LIGHTNING| P[Colors.Purple]
    T -->|WARNING| BR[Colors.Brand #FF4400]
    T -->|ERROR| BR
    style BR fill:#FF4400,color:#fff
Loading

Reviews (1): Last reviewed commit: "chore: rename changelog fragment" | Re-trigger Greptile

Comment thread app/src/main/java/to/bitkit/ui/components/ToastView.kt
@jvsena42
jvsena42 enabled auto-merge June 17, 2026 17:31

@ovitrif ovitrif left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utAck

@jvsena42
jvsena42 merged commit d2f6a6e into master Jun 17, 2026
29 of 32 checks passed
@jvsena42
jvsena42 deleted the fix/error-toast-color branch June 17, 2026 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Align .error toast color with design prototype

2 participants